Skip to content

Conversation

@blkt
Copy link
Contributor

@blkt blkt commented Sep 11, 2025

Description

This change adds a small section under the "Run MCP servers" page documenting how to override filter names and descriptions.

Fixes #127

Related issues/PRs

stacklok/toolhive#1841

Merge checklist

  • ToolHive CLI release

Reviews

  • Content has been reviewed for technical accuracy
  • Content has been reviewed for spelling, grammar, and style

@blkt blkt self-assigned this Sep 11, 2025
@vercel
Copy link

vercel bot commented Sep 11, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
docs-website Ready Ready Preview Comment Sep 12, 2025 0:27am

@blkt blkt requested a review from danbarr September 11, 2025 16:19
@danbarr danbarr requested a review from Copilot September 11, 2025 16:20
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This change adds documentation for the new --tools-override flag in the ToolHive CLI, which allows users to modify tool names and descriptions when running MCP servers. This helps users customize how tools are exposed to clients, which can be useful for network policy compliance or to influence agent behavior.

  • Adds a new "Override tool names and descriptions" section explaining the feature
  • Provides a concrete JSON configuration example using the fetch MCP server
  • Includes important guidance on combining tool filtering with tool overrides

Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @blkt. I ran it through Claude for a review and it had some good suggestions for clarity, I also had some ideas I applied; it seems Copilot has over-rotated on its instructions and is only flagging simple typos now in its reviews 🤦

@blkt
Copy link
Contributor Author

blkt commented Sep 12, 2025

@danbarr thank you for your thorough review, I should have addressed all comments.

"Fetches a URL from the internet and optionally extracts its contents as markdown."
```

You can now change it by supplying a file like the following
Copy link
Collaborator

@danbarr danbarr Sep 12, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I forgot I had a question about this, I assume the format is one entry under toolsOverride per tool? If so:

Suggested change
You can now change it by supplying a file like the following
To override this, create a configuration file with one entry under
`toolsOverride` for each tool you want to modify:

This change adds a small section under the "Run MCP servers" page
documenting how to override filter names and descriptions.

Fixes #127
Copy link
Collaborator

@danbarr danbarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @blkt, let's merge after the release happens.

@blkt blkt merged commit 84f13e4 into main Sep 12, 2025
6 checks passed
@blkt blkt deleted the docs/add-docs-for-tools-override branch September 12, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add documentation about Tool Override

3 participants